Fix: overlaping of "Sign in as a different user" button for small and medium devices#3438
Fix: overlaping of "Sign in as a different user" button for small and medium devices#3438bikalsiwakoti wants to merge 2 commits intodubinc:mainfrom
Conversation
|
@bikalsiwakoti is attempting to deploy a commit to the Dub Team on Vercel. A member of the Team first needs to authorize it. |
|
|
📝 WalkthroughWalkthroughThese changes fix mobile layout issues in the onboarding flow by converting a fixed-positioned component to use responsive positioning and adjusting padding distribution across screen breakpoints. Only CSS/styling modifications are made; no runtime logic or event handling changes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
2 participants
Fixes: #3431
Description:
On small and medium screen sizes, the component was overlapping with buttons and text due to position: fixed.
To fix this, I removed position: fixed specifically for small and medium devices and centered the component instead.
Before:
This ensures:
Only small and medium device styles were updated; desktop layout remains unchanged.
After:
I think it looks better in the center and just below of the "Create Workspace" button.
Just let me know if i can make it better.
Summary by CodeRabbit